You have used all Bugbot PR reviews included in your free trial for your GitHub account on this workspace.

To continue using Bugbot reviews, enable Bugbot for your team in the Cursor dashboard.

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Mar 17, 2026 11:14pm

Greptile Summary

This PR fixes a race condition where stale placeholder data from the previous workspace briefly populated the workflow registry during workspace switching, and also cleans up two related issues: missing beginMetadataLoad for direct URL navigation and overly broad cache invalidation in useCreateWorkspace.

Key changes:

• hooks/queries/workflows.ts: Guards completeMetadataLoad with !query.isPlaceholderData so the workflow registry is only populated with freshly-fetched data. Adds isPlaceholderData as a trigger for beginMetadataLoad to correctly display the loading state when TanStack Query serves placeholder data (the keepPreviousData path on workspace switch).
• hooks/queries/workspace.ts: Replaces the broad workspaceKeys.all invalidation in useCreateWorkspace.onSuccess with targeted workspaceKeys.lists() + workspaceKeys.adminLists() invalidation. The previous removeQueries calls on newly created workspace's detail/settings/permissions/members keys are also removed (they were no-ops since those keys would never have been cached for a brand-new workspace).
• lib/copilot/client-sse/handlers.ts: Calls abortAllInProgressTools at stream_end to ensure any tool calls still marked as in-progress are resolved when the SSE stream terminates, preventing tool calls from remaining stuck in an executing state.

Confidence Score: 4/5

• This PR is safe to merge — the changes are targeted, logically sound, and address a real UX bug with no apparent regressions.
• The workflows.ts and workspace.ts changes are straightforward and correct: the isPlaceholderData guard precisely solves the stale-data corruption, and the narrowed invalidation avoids unnecessary cache busting. The handlers.ts change is also sensible — abortAllInProgressTools already exists and is used from cleanupActiveState, so calling it from stream_end is a consistent cleanup path. Score is 4 rather than 5 primarily because no tests were added or updated to cover these code paths, leaving the correctness of the copilot stream-end cleanup unverified by automated tests.
• No files require special attention; all changes are well-scoped and do not touch security-sensitive paths.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant User
    participant useWorkflows
    participant TanStackQuery as TanStack Query
    participant Registry as Workflow Registry

    User->>useWorkflows: Switch workspace (workspaceId changes)
    Note over TanStackQuery: keepPreviousData kicks in
    TanStackQuery-->>useWorkflows: status='success', isPlaceholderData=true (stale data from old workspace)

    Note over useWorkflows: BEFORE fix: completeMetadataLoad fires<br/>with stale data → corrupt registry

    Note over useWorkflows: AFTER fix:
    useWorkflows->>Registry: beginMetadataLoad(newWorkspaceId) [isPlaceholderData=true]
    Note over Registry: Loading state shown

    TanStackQuery->>TanStackQuery: Fetch new workspace workflows
    TanStackQuery-->>useWorkflows: status='success', isPlaceholderData=false (real data)

    useWorkflows->>Registry: completeMetadataLoad(newWorkspaceId, realData)
    Note over Registry: Registry populated with correct workflows

_{Last reviewed commit: "fix(workspace): also..."}

@greptile